Skip to content

fix(cli): Don't add kotlin plugin for Cordova plugins#8518

Merged
jcesarmobile merged 2 commits into
nextfrom
RMET-5279
Jun 24, 2026
Merged

fix(cli): Don't add kotlin plugin for Cordova plugins#8518
jcesarmobile merged 2 commits into
nextfrom
RMET-5279

Conversation

@jcesarmobile

Copy link
Copy Markdown
Member

With Gradle 9, kotlin plugin is enabled, and causes issues to apply it, so remove all kotlin plugin related code from the CLI

Comment thread cli/src/android/update.ts Outdated
export async function handleCordovaPluginsGradle(config: Config, cordovaPlugins: Plugin[]): Promise<void> {
const pluginsGradlePath = join(config.android.cordovaPluginsDirAbs, 'build.gradle');
const kotlinNeeded = await kotlinNeededCheck(config, cordovaPlugins);
const kotlinVersionString = config.app.extConfig.cordova?.preferences?.GradlePluginKotlinVersion ?? '2.4.0';

@markemer markemer Jun 24, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This lint warning appears to be correct, Line 324 is the only place we use this.

@markemer markemer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just that one fix and I think we're good.

@jcesarmobile jcesarmobile requested a review from markemer June 24, 2026 13:57
@jcesarmobile jcesarmobile merged commit 9a2d3e6 into next Jun 24, 2026
6 checks passed
@jcesarmobile jcesarmobile deleted the RMET-5279 branch June 24, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants